Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compile statically against the MSVC CRT #3363

Merged
merged 1 commit into from Dec 14, 2016
Merged

Conversation

alexcrichton
Copy link
Member

This updates our AppVeyor builds to compile with -Ctarget-feature=+crt-static
to help Cargo be a bit more portable and not rely on the MSVC redistributable
artifacts. Over time this may even let us converge on only releasing one build
of Cargo and just pairing that with all Windows toolchains...

@rust-highfive
Copy link

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@bors
Copy link
Collaborator

bors commented Dec 13, 2016

☔ The latest upstream changes (presumably #3395) made this pull request unmergeable. Please resolve the merge conflicts.

This updates our AppVeyor builds to compile with `-Ctarget-feature=+crt-static`
to help Cargo be a bit more portable and not rely on the MSVC redistributable
artifacts. Over time this may even let us converge on only releasing one build
of Cargo and just pairing that with all Windows toolchains...
@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Collaborator

bors commented Dec 13, 2016

📌 Commit db60e6c has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Dec 13, 2016

⌛ Testing commit db60e6c with merge ff5ae77...

@bors
Copy link
Collaborator

bors commented Dec 13, 2016

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member Author

alexcrichton commented Dec 14, 2016 via email

@bors
Copy link
Collaborator

bors commented Dec 14, 2016

⌛ Testing commit db60e6c with merge ffa8b6e...

bors added a commit that referenced this pull request Dec 14, 2016
Compile statically against the MSVC CRT

This updates our AppVeyor builds to compile with `-Ctarget-feature=+crt-static`
to help Cargo be a bit more portable and not rely on the MSVC redistributable
artifacts. Over time this may even let us converge on only releasing one build
of Cargo and just pairing that with all Windows toolchains...
@bors
Copy link
Collaborator

bors commented Dec 14, 2016

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing ffa8b6e to master...

@bors bors merged commit db60e6c into rust-lang:master Dec 14, 2016
@bors bors mentioned this pull request Dec 14, 2016
@alexcrichton alexcrichton deleted the libcmt branch December 14, 2016 06:34
@pravic
Copy link

pravic commented Jun 20, 2017

Is there a way to specify a crt-static feature in Cargo.toml? Besides .cargo/config, which requires nightly in addition.

@alexcrichton
Copy link
Member Author

Not currently, no. It needs to be specified as part of RUSTFLAGS currently

@pravic
Copy link

pravic commented Jun 20, 2017

I see. Thanks anyway

@ehuss ehuss added this to the 1.15.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants